Repository navigation
chore: merge actions-vm build workflows; add setup-oci-cli action - #670
Merged
Merged
Conversation
periodic-build-new-actions-vm.yml and change-file-build-new-actions-vm.yml were the same 700-line pipeline differing only in trigger, the image-exists gate, GITHUB_PERIODIC, and an apt package list. Merge them into build-actions-vm.yml with schedule + push + workflow_dispatch triggers; the image-exists gate is applied only on schedule (or on dispatch unless force_rebuild is set). Add .github/actions/setup-oci-cli to replace the repeated setup-python + pip install oci-cli steps and the inline ~/.oci/config heredocs. It exports OCI_CONFIG_FILE and OCI_CLI_CONFIG_FILE so the OCI CLI, Packer, and the Go tools all find the same config. Signed-off-by: Jeffrey Sica <me@jeefy.dev>
github-actions
Bot
added
needs-triage
koksay
reviewed
Sep 10, 2026
Drop qemu-system-arm/qemu-efi-aarch64 from the x86 job; the arm image is built by the OCI builder in a separate job and does not use QEMU. Make the ~/.oci/config dependency explicit: name the setup-oci-cli step, reference its config-file output from the arm build and Enable A10 shapes steps, and assert the file exists before use. setup-oci-cli now fails if key-content is set but any other config field is empty. Signed-off-by: Jeffrey Sica <me@jeefy.dev>
koksay
approved these changes
Sep 11, 2026
Member
|
Triggered action after the merge: https://github.com/cncf/automation/actions/runs/34570462842 |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Part of an ongoing effort to clean up, consolidate, and prune this repo (follows #664, #668, #669).
build-actions-vm.ymlreplacesperiodic-build-new-actions-vm.yml+change-file-build-new-actions-vm.yml(1383 lines → 693). They were the same pipeline; the only deltas were:ci/gha-runner-vm*/**workflow_dispatch(force_rebuild)SKIP_IF_IMAGE_EXISTS= schedule, or dispatch withoutforce_rebuildGITHUB_PERIODICtrue${SKIP_IF_IMAGE_EXISTS}qemu-system-x86 xorriso+ qemu-system-arm qemu-efi-aarch64~/.local/bin~/bin.github/actions/setup-oci-cli— composite replacing 8×setup-python+pip install oci-cliand 3× inline~/.oci/configheredocs per file. Writes the config only whenkey-contentis passed, exportsOCI_CONFIG_FILE/OCI_CLI_CONFIG_FILEso the CLI, Packer, andgha-runner-vm's Go code (which readsOCI_CONFIG_FILE) all agree. Also adopted incleanup-custom-images.yaml.Not touched:
publish-cloudrunner-images.yml(writes a container-path config fordocker run -v) andci-test.yaml(builds a k8s Secret) — different shapes, follow-up.Can't be exercised on a fork (needs OCI secrets + self-hosted runners); YAML validated locally. Recommend a
workflow_dispatchrun withforce_rebuild=falseafter merge to confirm the skip path, then one nightly.